← Back to issue list

snapcraft prints the setup dir deprecated warning even if it's not a snapcraft dir

View original Launchpad issue

Metadata

Project
snapcraft (launchpad)
Number
#1676241
Type
issue
State
open
Author
~elopio
Labels
ui
Created
2017-03-27 00:40:26.525407+00:00
Updated
2017-11-20 22:33:27.813709+00:00
Closed

Current evaluation

No evaluation has been recorded for this issue yet.

Issue body

We moved the snap assets from setup/gui to snap/gui. So snapcraft started printing the following deprecation warning when it finds the setup dir: DEPRECATED: Assets in 'setup/gui' should now be placed in 'snap/gui'. See http://snapcraft.io/docs/deprecation-notices/dn3 for more information. However, the same warning is printed even if the setup dir doesn't contain any snapcraft assets. For example, the golem project has a setup dir in the root of the tree that has nothing to do with snapcraft, and it still gets the warning. This is confusing and ugly. We could easily improve it a little checking for the setup/gui dir, instead of just setup. Even better if we check for the contents of setup/gui before printing the warning. To reproduce: $ snapcraft init $ emacs snap/snapcraft.yaml # replace the part with: parts: my-part: source: . plugin: dump $ mkdir setup $ snapcraft

Evaluation history

No evaluation history available.